Summary

This PR removes 209 lines of unnecessary comments from the project's C++ source files to improve code cleanliness and maintainability.

Changes Made

Cleaned up 6 source files by removing:

• Commented-out old/debug code - Removed inactive Serial.println() debug statements and old function implementations
• Redundant inline comments - Eliminated comments that simply repeat what the code already clearly expresses
• Outdated commented-out implementations - Removed an entire old API implementation in gsm_manager.h that was no longer in use
• Trivial descriptive comments - Removed comments like // Module RTC and // Active l'heure réseau that don't add meaningful context

Files Modified

Examples of Removed Comments

Commented-out debug code:

// Before
sim800.println("AT+CLTS=1");
//Serial.println("Active l'heure");  // REMOVED
delay(1000);

// After  
sim800.println("AT+CLTS=1");
delay(1000);

Redundant inline comments:

// Before
#define RX 14 // GPIO14 = D5  // REMOVED
#define TX 12 // GPIO12 = D6  // REMOVED

// After
#define RX 14
#define TX 12

Outdated commented-out implementations:

// Removed 80+ lines of old commented-out getNowTime() function
// that was replaced by getNetworkTime()

Verification

• ✅ All files pass basic syntax validation (matched braces, proper structure)
• ✅ No functional code was modified - only comments were removed
• ✅ Code logic and behavior remain completely unchanged
• ✅ Net reduction: 254 deletions, 45 insertions (mostly whitespace cleanup)

This cleanup makes the codebase easier to read and maintain by removing visual clutter while preserving all important context.

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.